From: Florian Müllner <fmuellner@gnome.org>
Date: Fri, 15 Feb 2013 15:47:39 +0000 (+0100)
Subject: appDisplay: Set a maximum column number
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~4739
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=2fbd9b95a74bf9dc12ef9c0fe566301246379b9c;p=gnome-shell.git

appDisplay: Set a maximum column number

In particular on widescreen monitors, the number of items in a row
can be overwhelming, so limit the number of columns and center the
view.

https://bugzilla.gnome.org/show_bug.cgi?id=694192
---

diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 456feb563..da876401b 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -27,6 +27,7 @@ const Util = imports.misc.util;
 const MAX_APPLICATION_WORK_MILLIS = 75;
 const MENU_POPUP_TIMEOUT = 600;
 const SCROLL_TIME = 0.1;
+const MAX_COLUMNS = 6;
 
 // Recursively load a GMenuTreeDirectory; we could put this in ShellAppSystem too
 function _loadCategory(dir, view) {
@@ -51,7 +52,8 @@ const AlphabeticalView = new Lang.Class({
     Name: 'AlphabeticalView',
 
     _init: function() {
-        this._grid = new IconGrid.IconGrid({ xAlign: St.Align.START });
+        this._grid = new IconGrid.IconGrid({ xAlign: St.Align.MIDDLE,
+                                             columnLimit: MAX_COLUMNS });
 
         this._appIcons = {}; // desktop file id
         this._allApps = [];
@@ -62,6 +64,7 @@ const AlphabeticalView = new Lang.Class({
         this.actor = new St.ScrollView({ x_fill: true,
                                          y_fill: false,
                                          y_align: St.Align.START,
+                                         x_expand: true,
                                          style_class: 'vfade' });
         this.actor.add_actor(box);
         this.actor.set_policy(Gtk.PolicyType.NEVER, Gtk.PolicyType.AUTOMATIC);
@@ -140,7 +143,7 @@ const AppDisplay = new Lang.Class({
                                   x_fill: true, y_fill: true });
 
         this._view = new AlphabeticalView();
-        box.add(this._view.actor);
+        box.add(this._view.actor, { expand: true });
 
         // We need a dummy actor to catch the keyboard focus if the
         // user Ctrl-Alt-Tabs here before the deferred work creates